Skip to content

Removes codacy#86

Merged
mdellabitta merged 1 commit intomainfrom
remove-codacy
Dec 27, 2024
Merged

Removes codacy#86
mdellabitta merged 1 commit intomainfrom
remove-codacy

Conversation

@mdellabitta
Copy link
Contributor

@mdellabitta mdellabitta commented Dec 27, 2024

Important

Remove Codacy integration from CI workflow and README.

  • CI Workflow:
    • Removes Codacy Coverage Reporter step from .github/workflows/scala.yml.
  • Documentation:
    • Removes Codacy Quality and Coverage badges from README.md.

This description was created by Ellipsis for acf6391. It will automatically update as commits are pushed.

@mdellabitta mdellabitta changed the title ``Removes codacy Removes codacy Dec 27, 2024
Copy link

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Looks good to me! Reviewed everything up to acf6391 in 17 seconds

More details
  • Looked at 24 lines of code in 2 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. .github/workflows/scala.yml:23
  • Draft comment:
    Consider removing coverage and coverageReport from the sbt command if Codacy is no longer used for coverage reporting.
  • Reason this comment was not posted:
    Comment was on unchanged code.

Workflow ID: wflow_XmPVpmgrQZSuwURA


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@mdellabitta mdellabitta reopened this Dec 27, 2024
@mdellabitta mdellabitta merged commit 3d51d7c into main Dec 27, 2024
5 checks passed
@mdellabitta mdellabitta deleted the remove-codacy branch December 27, 2024 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant